Skip to content

Change web3name in docs #307

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
May 28, 2024

Conversation

ChrisChinchilla
Copy link
Contributor

fixes KILTProtocol/ticket#3156

Please include a summary of the changes provided with this pull request and which issue has been fixed.
Please also provide some context if necessary.

@ChrisChinchilla ChrisChinchilla force-pushed the 3156-john_does-credentials-are-revoked branch from 9c1057a to 4bb0431 Compare April 2, 2024 14:56
Signed-off-by: Chris Chinchilla <chris@kilt.io>
@ChrisChinchilla ChrisChinchilla force-pushed the 3156-john_does-credentials-are-revoked branch from 4bb0431 to 193fd5e Compare April 4, 2024 09:09
Signed-off-by: Chris Chinchilla <chris@kilt.io>
@ChrisChinchilla ChrisChinchilla marked this pull request as ready for review April 18, 2024 11:38
@ChrisChinchilla
Copy link
Contributor Author

@Dudleyneedham I changed to "kiltnerd123" and this kind of works, but two things confuse me… The verification code in /getting_started/06_verify_credential.ts fails and I am not quite sure why… I removed code that tests the same on spirit net, as I didn't have enough tokens to set up the DID on spirit net, and I wasn't sure if it was necessary anyway?

@ntn-x2
Copy link
Member

ntn-x2 commented May 24, 2024

Yeah I also think testing for Spiritnet is not required. Can you please update the PR and resolve the conflicts?

Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-307.d3oeht7cvddn19.amplifyapp.com

@ChrisChinchilla ChrisChinchilla requested a review from ntn-x2 May 28, 2024 11:17
@ChrisChinchilla
Copy link
Contributor Author

@ntn-x2 Done, but I am a bit stuck on failures that have now appeared. If you have any ideas, let me know and I will dig in…

@ChrisChinchilla ChrisChinchilla merged commit 2d9a7f7 into master May 28, 2024
5 of 6 checks passed
@ChrisChinchilla ChrisChinchilla deleted the 3156-john_does-credentials-are-revoked branch May 28, 2024 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants